-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added interfaces and some generic modeling blocks. #47
Open
cfelton
wants to merge
33
commits into
master
Choose a base branch
from
gsocend
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Organized many of the test support to jpegenc/testing. Also adding models and model tests.
This commit contains the addition of general processing element models (ProcessingSubblock). This commit also has some other minor changes made during test creation.
Added a DCT data flow test to emulate the possible data flow through the system. This commit also has changed to the interfaces, the previous simulation `assign` functions were replaced with the `next` attribute. The `assign` function is now a myhdl.block to be used in the convertible HDL blocks.
The Verilog cosimulation tests, reference design tests, were moved to a separate directory. This was done to organize the tests. The cosim (ref tests) being in a separate directory can be troublesome. The cosim needs to know the path to the Verilog files and the VPI module. When run from pytest this typically will be from the test_jpeg/tests/ directory, all the paths would be relative from here. Currently all the paths are fixed from the tests/ dir, this should be enhanced in the future to determine which directory the pytest runner exectuing from and adjust the paths accordingly.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This merge contains quite a few changes. It includes reorganization of many of the files. It also include the addition of interfaces to address issue #46. The interfaces have not been incorporated into the subblocks. Generic modeling blocks were created to test and model the interfaces. The generic processing blocks allow experimentation and prototyping of different data flows to support HD compression.
The documents have also been reorganized. The documents are still a work in progress. Most of the existing documents are reference for the various subblocks. Additional documentation needs to be added: design docs (why things were done they way they were); user docs; etc.
The documents now contain a basic overview.